SkinTemplate: Move bottomScripts() back sightly
Follows-up
80e5b160e,
95fc8a96db.
In
80e5b160e, the module queue formatting was changed and was now fixed at
the first call to headElement() or bottomScripts(), instead of being kept
dynamic in between. Doing so fixed various bugs but also broke a few cases
where things still add to the bottom queue after the headElement() is
already created (such as sidebar).
In
95fc8a96db, the bottomScripts() method was moved to the very bottom of
prepareQuickTemplate(). This was imho too far and not needed because any line
after 'headElement()' element is effectively the same since that will commit
the queue.
It also changed the order of hook execution, from:
- SkinAfterBottomScripts, SkinTemplateOutputPageBeforeExec, SkinAfterContent
to:
- SkinTemplateOutputPageBeforeExec, SkinAfterContent, SkinAfterBottomScripts
With the latter no longer being able to add things to the module queue since
it now runs after headElement() commits the queue.
Move it back up, to right above headElement(). This way the order is the
same as before
95fc8a96db (first bottomScripts, then headElement). But still
after all other properties, thus still keeping the regression from
80e5b160e
fixed.
Bug: T87871
Change-Id: I8f774880e60e31c0e996b20acdc5ce5e42a84b01